Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

11.0 implemted printing invoice lines in receipts #1186

Closed
wants to merge 2 commits into from
Closed

11.0 implemted printing invoice lines in receipts #1186

wants to merge 2 commits into from

Conversation

em230418
Copy link
Contributor

itpp-bot
itpp-bot reviewed Dec 17, 2019
View reviewed changes
Copy link
Contributor

@itpp-bot itpp-bot left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 installable module is updated:

└─ pos_invoice_pay/
   ├─ __manifest__.py
   ├─ doc/
   |  └─ changelog.rst
   ├─ models.py
   ├─ report/
   |  └─ report.py
   ├─ static/
   |  └─ src/
   |     └─ xml/
   |        └─ pos.xml
   └─ view.xml

Not installable modules remain unchanged.

sent by ✌️ Odoo Review Bot

pos_invoice_pay/doc/changelog.rst

**New:** Implement printing invoice lines in receipts
**Fix:** Fixed field properties typo in sale_order_cashier_selection and invoice_cashier_selection

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

KolushovAlexandr
KolushovAlexandr suggested changes Dec 17, 2019
View reviewed changes
Copy link
Contributor

@KolushovAlexandr KolushovAlexandr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The receipt is not updated

@em230418 em230418 force-pushed the 11.0-pos_invoice_pay-issue1182 branch 2 times, most recently from 0eaa2f5 to 5fa969b Compare December 19, 2019 07:47
@KolushovAlexandr
Copy link
Contributor

Eugene said orderlines are cut. Confirm. Then only style fixes are needed

@trojikman trojikman force-pushed the 11.0-pos_invoice_pay-issue1182 branch from 5fa969b to 739dc78 Compare June 8, 2020 22:10
@trojikman
Copy link
Contributor

trojikman commented Jun 8, 2020 •
edited
Loading

@yelizariev I'll fix it in separate PR

@trojikman
Copy link
Contributor

Screenshot from 2020-06-09 03-42-20

@trojikman trojikman requested a review from yelizariev June 8, 2020 22:43
yelizariev
yelizariev approved these changes Jun 9, 2020
View reviewed changes
pos_invoice_pay/doc/changelog.rst

**New:** Implement printing invoice lines in receipts
**Fix:** Fixed field properties typo in sale_order_cashier_selection and invoice_cashier_selection
**Fix:** Fixed "'bool' object has no attribute 'mapped'" error in get_sale_details
Copy link
Collaborator

@yelizariev yelizariev Jun 9, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Описания фиксов слишком техническое. Надо писать когда они происходят, а не в каких функциях

@em230418
Copy link
Contributor Author

em230418 commented Jun 9, 2020

Нельзя принимать.При печати в POS-принтер там режутся строки на середине и я так и не исправил.

@em230418 em230418 marked this pull request as draft June 9, 2020 06:22
@em230418
Copy link
Contributor Author

em230418 commented Jun 9, 2020

Задача описана тут: https://www.it-projects.info/web#id=2975&view_type=form&model=project.task

Там в комментариях см. обсуждения на 18-ое декабря 2019.

@em230418 em230418 closed this by deleting the head repository Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itpp-bot itpp-bot itpp-bot left review comments

@Ramil-Mukhametzyanov Ramil-Mukhametzyanov Ramil-Mukhametzyanov approved these changes

@KolushovAlexandr KolushovAlexandr KolushovAlexandr approved these changes

@trojikman trojikman trojikman approved these changes

@yelizariev yelizariev yelizariev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@em230418 @KolushovAlexandr @trojikman @yelizariev @Ramil-Mukhametzyanov @itpp-bot